Skip to content

Feat allign functionality for path tool across x and y axis#3619

Open
Ayush2k02 wants to merge 10 commits intoGraphiteEditor:masterfrom
Ayush2k02:feat-allign-buttons-on-path-tool
Open

Feat allign functionality for path tool across x and y axis#3619
Ayush2k02 wants to merge 10 commits intoGraphiteEditor:masterfrom
Ayush2k02:feat-allign-buttons-on-path-tool

Conversation

@Ayush2k02
Copy link
Contributor

@Ayush2k02 Ayush2k02 commented Jan 10, 2026

POINTS

Screen.Recording.2026-01-10.at.1.14.13.PM.mov

SEGMENTS

Screen.Recording.2026-01-10.at.1.42.41.PM.mov

DISCLAIMER : Claud Code - cli extension was used in the PR ( this is completely free for university students )

CLOSES #3375

responses.add(GraphOperationMessage::Vector { layer, modification_type });
}

// TODO: figure out this Special case: When exactly 2 anchors are selected, skip handle transformations
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a TODO here based on this discussion

#3375 (comment)

@Ayush2k02
Copy link
Contributor Author

Also, which formatter do you guys use, all my builds are failing due to this

@Keavon
Copy link
Member

Keavon commented Jan 10, 2026

That's covered in the contributor guide. Now that you've gotten some understanding of things, I recommend giving it a second read so you can take in the specifics like that and others.

@Ayush2k02
Copy link
Contributor Author

Done, have updated this PR and my other PRs with the contributor guidelines !

@timon-schelling
Copy link
Member

!build

@github-actions
Copy link

📦 Build Complete for f33939e
https://e4373aa3.graphite.pages.dev

@Ayush2k02
Copy link
Contributor Author

Any updates on this guys !

@Ayush2k02 Ayush2k02 marked this pull request as draft February 2, 2026 18:35

impl AlignAggregate {
/// Given a bounding box `[min, max]`, returns the alignment target position.
pub fn target_position(self, bbox: [DVec2; 2]) -> DVec2 {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what to name so just named it as target_position ( min , max or center ) , open to other suggestions as well

@Ayush2k02 Ayush2k02 marked this pull request as ready for review February 3, 2026 04:06
@Ayush2k02
Copy link
Contributor Author

Screen.Recording.2026-02-03.at.9.47.39.AM.mov

Here is the recording after the recent commits

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor refactor to use the shared utility function here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Alignment buttons for points in the Path tool control bar

3 participants